add 'commit_order' option for margin popup #1811
Conversation
|
I'm not opposed to this, but when I try opening the kernel in neogit it opens quite quickly. How long does it take you to open? |
|
This is my workflow: git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git neogit_test
cd neogit_test
echo hello > hello & git add hello
neovim
:NeogitIt takes me approximately 21 seconds before I can see the staged changes. Any subsequent change like |
|
Well, that is objectively terrible 😮💨 If this helps you bring that time down, then I'll call that a win. |
|
@CKolkey thanks! |
No worries! Thank you for the PR :) |
Oof, sorry about the performance hit! Should have thought about allowing a config override. Thanks for adding it! |
3 participants
PR #1794 introduced a default --topo-order for the margin popup. This makes Neogit’s status view nearly unusable due to the cost of topological sorting on repos with a large commit history (like the kernel).
This PR adds a commit_order config option (default "topo") so users can override or disable the behavior (commit_order = ""). That restores fast refreshes while still allowing topological or date-based ordering when desired.